Skip to content

Comments

feat: add MESSENGER_CONSUMER_NAME env var#176

Merged
Kevin Rudde (kevinrudde) merged 2 commits intomainfrom
add-messenger-consume-name
Feb 23, 2026
Merged

feat: add MESSENGER_CONSUMER_NAME env var#176
Kevin Rudde (kevinrudde) merged 2 commits intomainfrom
add-messenger-consume-name

Conversation

@kevinrudde
Copy link
Member

@kevinrudde Kevin Rudde (kevinrudde) commented Feb 23, 2026

This pull request updates the environment variables for worker containers and improves the related test coverage. The main changes involve adding a new environment variable to expose the consumer name and modifying the Redis DSN format for workers.

Environment variable improvements:

  • Added a new environment variable MESSENGER_CONSUMER_NAME for worker containers, which is dynamically set to the Pod's metadata name using a field reference. [1] [2]

Redis DSN updates:

  • Updated the MESSENGER_TRANSPORT_DSN value for workers to remove the /consumer segment from the Redis DSN path, aligning it with the expected format. [1] [2]

Test enhancements:

  • Updated tests in store_test.go to check for the presence of the new MESSENGER_CONSUMER_NAME environment variable and to verify the modified Redis DSN format. [1] [2]

@kevinrudde Kevin Rudde (kevinrudde) merged commit bbfefe4 into main Feb 23, 2026
5 checks passed
@kevinrudde Kevin Rudde (kevinrudde) deleted the add-messenger-consume-name branch February 23, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants